Skip to content
This repository was archived by the owner on Sep 2, 2021. It is now read-only.

Changing Permission for xdg-open file to 755. xdg-open needs to be execu... - #505

Merged
jasonsanjose merged 1 commit into
linux-1547from
prafulVaishnav/fix-10229
Feb 24, 2015
Merged

jasonsanjose merged 1 commit into
linux-1547from
prafulVaishnav/fix-10229

Conversation

@prafulVaishnav

Copy link
Copy Markdown
Collaborator

...table otherwise browser does not launch for MultiBrowser Preview. grunt-contrib-copy does not preserve permissions so setting it explicitly for linux. Issue - adobe/brackets#10229.

…ecutable otherwise browser does not launch for MultiBrowser Preview. grunt-contrib-copy does not preserve permissions so setting it explicitly for linux.
@prafulVaishnav

Copy link
Copy Markdown
Collaborator Author

@ingorichter @jasonsanjose Can you please review the PR. Merging it into linux-1547 as we will be using it for generating linux build.

@jasonsanjose

Copy link
Copy Markdown
Member

Merging.

jasonsanjose added a commit that referenced this pull request Feb 24, 2015
Changing Permission for xdg-open file to 755. xdg-open needs to be execu...
@jasonsanjose
jasonsanjose merged commit 56330f8 into linux-1547 Feb 24, 2015
@jasonsanjose
jasonsanjose deleted the prafulVaishnav/fix-10229 branch February 24, 2015 21:29
@peterflynn

Copy link
Copy Markdown
Member

@prafulVaishnav We should also submit this change to master so it doesn't get lost when we cut over to the new CEF on Linux. Can you put up a PR for master too?

@prafulVaishnav

Copy link
Copy Markdown
Collaborator Author

@peterflynn Submitted the PR #506. Can you review and merge it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants